-
-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ignore duplicate rpaths in #change_rpath
#438
Conversation
Even the existing test that I've just deleted seems to differ from
|
This causes a deviation in behaviour from `install_name_tool`, which will happily change an existing rpath to one that already exists. Closes Homebrew#436.
42714f6
to
3275fec
Compare
Hmm, I wonder if |
I was wondering that too! But I don't have access to an old Mac to check. |
Neither do I anymore, unfortunately. I remember we did quite a bit of behavioral testing against |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
Have you had the chance to think about how you want to handle this, @woodruffw? |
Sorry for the delay! And yeah: I think I agree that this change is for the best. |
No apologies necessary, and thanks! |
This causes a deviation in behaviour from
install_name_tool
, whichwill happily change an existing rpath to one that already exists.
Closes #436.